-
Notifications
You must be signed in to change notification settings - Fork 42
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix the issue with assigning kernels to ops before registering kernels on macOS #582
Conversation
Hello. You may have forgotten to update the changelog!
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #582 +/- ##
==========================================
- Coverage 98.94% 98.68% -0.27%
==========================================
Files 167 167
Lines 24527 23959 -568
==========================================
- Hits 24269 23644 -625
- Misses 258 315 +57 ☔ View full report in Codecov by Sentry. |
Fix Issue PennyLaneAI/pennylane-lightning#583 in Catalyst **temporary** until we merge PR PennyLaneAI/pennylane-lightning#582 [sc-52500]
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the find @maliasadi
pennylane_lightning/core/src/simulators/lightning_qubit/gates/DynamicDispatcher.hpp
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me! Thanks @maliasadi !
Update Lightning tags to PennyLaneAI/pennylane-lightning@489cff7 as the fix PennyLaneAI/pennylane-lightning#582 is now merged into the master branch. PS. This is related to the "temporary" fix in #412
The addition of arbitrary controlled operations in
lightning.qubit
(PR #516) breaks the use of thelightning.qubit
C++ API in Catalyst on macOS. This PR fixes the following issue:Fixes #583
[sc-52500]